home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c-part1 / 4348 < prev    next >
Encoding:
Internet Message Format  |  1996-08-05  |  2.2 KB
  1. Path: mudskipper.cac.psu.edu!user
  2. From: fcusack@tdx.org (frank.)
  3. Newsgroups: comp.lang.c
  4. Subject: Re: simple code, argc, argv, strcmp()
  5. Date: Sat, 03 Feb 1996 16:45:28 -0400
  6. Organization: Penn State University, Center for Academic Computing
  7. Message-ID: <fcusack-0302961645280001@mudskipper.cac.psu.edu>
  8. References: <11f7cc$17261a.3b3@daprez> <4etj7c$bma@news.iag.net> <fcusack-0202961621470001@mudskipper.cac.psu.edu>
  9. NNTP-Posting-Host: mudskipper.cac.psu.edu
  10.  
  11. In article <fcusack-0202961621470001@mudskipper.cac.psu.edu>,
  12. fcusack@tdx.org (frank.) wrote:
  13.  
  14. > In article <4etj7c$bma@news.iag.net>, jatmon@iag.net (John R Buchan) wrote:
  15. > 
  16. > > In article <11f7cc$17261a.3b3@daprez>, otisg@panther.middlebury.edu says...
  17. > > >
  18. > > >
  19. > > >#include <stdio.h>
  20. > > >#include <stdlib.h>
  21. > > >#include <string.h>
  22. > > >
  23. > > >int main (int argc, char **argv) {
  24. > > >
  25. > > >  void Usage   (void);
  26. > > >  void Encode (int, char **);
  27. > > >  void Decode (int, char **);
  28. > > >
  29. > > >  if (!strcmp(argv[1],"-d") || !strcmp(argv[1],"-e")) {
  30. > > >    Usage();
  31. > > >  }
  32. > > <snip>
  33. > > 
  34. > > I assume you mean this to call Usage if argv[1] is not "-e" and not "-d"?
  35. > > Change the || to &&.  
  36. > 
  37.  
  38. I mistakenly wrote:
  39.  
  40. > The && test would _never_ pass. argv[1] could never be both "-d" and "-e".
  41. > The || is correct here.
  42. > 
  43.  
  44. Yes, the && test would never pass, but the || test is bad also. The ||
  45. will pass if you pass "-d" or "-e". So then you will call Usage()
  46. incorrectly. The test should be:
  47.  
  48.    if (!(!strcmp(argv[1], "-d") || !strcmp(argv[1], "-e"))) {
  49.       Usage();
  50.    }
  51.  
  52. > >Of course, if no argument was passed, who knows what
  53. > > will happen here.  You should always test argc first to be certain that 
  54. > > the argv element exists.
  55. > 
  56.  
  57. and I also incorrectly wrote:
  58.  
  59. > argv[1] is always guaranteed to exist. It is simply NULL if there were no
  60. > arguments.
  61.  
  62. Dan Pop has the poop on this one in his followup. eg, on the Mac, argc is
  63. probably always zero, thus argv[1] is undefined.
  64. ~Frank
  65.  -- I am Pentium of Borg.  Division is futile.  You will be approximated. --
  66.  --   If you build it, they will come --> http://www.tdx.org/~fcusack/    --
  67.  -- PGP key fingerprint: 01 C0 C0 B9 CC 78 67 0F  3F 64 80 65 8B 0F F9 EA --
  68.